JDBCType BIT for boolean classes

I'm bringing this as curiosity because i'm not entirely sure if its an issue.   I was recently using a JDBC driver for a proprietary system and their driver does not support the BIT JDBC type.   Therefore, when using boolean fields, it would fail.  It made me look into it and noticed that your mappings of Java classes to JDBC Type had BIT for boolean/Boolean as you can see here:
https://github.com/spring-projects/spring-data-relational/blob/967f0f9179e2c8cbdbfbfd2b633362c6b532379b/spring-data-jdbc/src/main/java/org/springframework/data/jdbc/support/JdbcUtil.java#L77
However, the Spring framework JDBC core library seems to map Boolean/boolean to Types.BOOLEAN.  You can see here:
https://github.com/spring-projects/spring-framework/blob/main/spring-jdbc/src/main/java/org/springframework/jdbc/core/StatementCreatorUtils.java#L92
One obvious solution is to ask the proprietary company to support BIT type.  I am curious if the BIT for boolean/Boolean deviation was purposeful.  As a work around, i'm currently using reflection to remap the types as its a small service and the impact of this change seems rather low.   Am I missing any other impact from doing this?
    Class<?> clazz = JdbcUtil.class;
    Field field = clazz.getDeclaredField("sqlTypeMappings");
    field.setAccessible(true);
    Map<Class<?>, SQLType> map = (Map<Class<?>, SQLType>) field.get(null);
    map.put(Boolean.class, JDBCType.BOOLEAN);
    map.put(boolean.class, JDBCType.BOOLEAN);

Type mapping is a concern on its own as there is a common set that applies to most databases. However, things deviate in the details as some databases do not have boolean types or bit types, or these would simply suggest using a specific type to map a certain Java type.
JdbcUtil is a static utility that doesn't allow for external customization.
Zooming out, we derive column types based either on their Java type or, as an alternative, on a registered custom converter that can return JdbcValue holding a SQLType.
You could also register a converter for your Boolean data types.
Generally, we currently do not support type hints on a per-property level (something like @Column(type = …)). Also, it would make sense to refine our typing information system for extensibility and consider a bit of the type mapping in our dialects.

Am I missing any other impact from doing this?

All boolean types will map into BIT. Assuming you're using the same database for all entities, that should be fine.

That sounds good, thanks!  A converter definitely works for me.  It would be nice at a property level, but I'm glad i had a solution outside of reflection!

#746 is related.

Note in this use case because the mapping is BIT for boolean values, Spring JDBC Template for setting arguments on a PreparedStatement ends up falling back to setObject here instead of the more specific setBoolean. This makes the support a bit variable depending on the JDBC driver implementation of setObject. It's unclear if maybe Spring JDBC Template should be modified to try to detect this scenario to call setBoolean instead or if the type mappings here should change from BIT ==> BOOLEAN.
I.e. in theory Spring JDBC template could do something like:
If (sqlType == BIT && inValue instanceof Boolean) {
     // use setBoolean
}

around here


Emotecraft + Just expressions.

Just find out how to make Just Expressions work with Emotecraft.
Simply delete body parts in player.jem and player_slim.jem file.
.\assets\minecraft\optifine\cem then scroll down to "part": "body" and delete everything. Should be looking like this:

Is it possible to make Emotecraft work with Fresh Moves?

No, Emotecraft use own part system that have bend parts.

Is it possible to make Emotecraft work with Fresh Moves?

No. Emotecraft use their own parts system that have bends.


Support for laravel 6.x changes

need chahge composer.json
from
"illuminate/support": "^5.5",
to
"illuminate/support": ">=5.5",

Thank you!


CONTENT-7377: fixing DB connections leaks or possible leaks.

Changes looks good and merged into development branch.


Unable to provision SharePoint Success Site

Not getting a correlation ID though. Using the following URL (direct link from lookbook site):
https://lookbook.microsoft.com/details/0b860749-56a0-4c4c-992c-536d56d9accf
When hitting "Add to your tenant" I'm getting the following error on the next page:
Sorry, something went wrong
There was an unexpected issue.
Could not load file or assembly 'OfficeDevPnP.Core, Version=3.28.2012.0, Culture=neutral, PublicKeyToken=5e633289e95c321a' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)

Hi,
Thanks for your feedback. The issue with the SharePoint Success Site is now fixed. Please, try again and in case of any failure, create another issue.

Hi,
Thanks for your feedback. The issue with the SharePoint Success Site is now fixed. Please, try again and in case of any failure, create another issue.


gym-http-api should reference the official OpenAI submodule

Currently, submodule pulls from https://github.com/korymath/gym-http-api should be shifted to https://github.com/openai/gym-http-api now that it includes all necessary Lua bindings.

Closed with #19


Youtube age-restricted video with disabled embeds won't download

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2019.07.02
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.youtube.com/watch?v=I7gtoLmHd5w', '--username', 'PRIVATE', '--password', 'PRIVATE', '-v']
[debug] Encodings: locale cp1252, fs mbcs, out cp1252, pref cp1252
[debug] youtube-dl version 2019.07.02
[debug] Python version 3.4.4 (CPython) - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-92574-g72b047a7a7, ffprobe N-92574-g72b047a7a7
[debug] Proxy map: {}
[youtube] Downloading login page
[youtube] Looking up account info
[youtube] Logging in
WARNING: Unable to log in: HTTP Error 400: Bad Request
[youtube] I7gtoLmHd5w: Downloading webpage
[youtube] I7gtoLmHd5w: Downloading embed webpage
[youtube] I7gtoLmHd5w: Refetching age-gated info webpage
ERROR: Content Warning
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmphnjfkuku\build\youtube_dl\YoutubeDL.py", line 796, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmphnjfkuku\build\youtube_dl\extractor\common.py", line 530, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmphnjfkuku\build\youtube_dl\extractor\youtube.py", line 2097, in _real_extract
youtube_dl.utils.ExtractorError: Content Warning
--

Description

I can't download this video: https://www.youtube.com/watch?v=I7gtoLmHd5w
It's age-restricted and embeds are disabled (checked by changing URL with the /embed/ trick).
I tried both without and with the --username and --password options. I am using the most up to date version released.
Getting the error noted in verbose log.

@dstftw dstftw
IF your going to close an issue, please provide context for the reason you closed it, this includes providing issue numbers to what you perceive are duplicate issues. With over 21600 issues their's a lot and people MAY NOT BE able to find the same issues that you find that you consider duplicate issues OF OPENED issues.
Thank you.

@dstftw Yeah this isn't helpful at all, can you please reference the issue that you think this is a duplicate of? I couldn't find it.

I found two similar issues (#15184, #17216) on the first page of my search (took me literally no more than 15 seconds). I wonder why so many people find it so hard to discover what they are looking for. BTW have you tried --cookies ?

@TomArrow Issue #11270 appears to be the root of the username/login issue.  Also @bitraid's suggestion works.  Get https://addons.mozilla.org/en-US/firefox/addon/cookies-txt/ and export your current cookies with it to a cookies.txt file  (FWIW: I had to restart Firefox before this addon's toolbar button would actually work).  Then add "--cookies c:\your_path_here\cookies.txt" to your youtube-dl command line.
@dstftw youtube-dl can be the reason people come to Github for the first time. I wonder if to cut down on duplicates (and therefore your fatigue factor), you should change this text in your Checklist from:
[x]  I've searched the bugtracker for similar issues including closed ones
to something that is more instructive for newcomers, like:
[x]  I've searched the bugtracker for similar issues that are open
[x]  I've searched the bugtracker for similar issues that are closed (by changed the default "is:issue is:open" text in the search field to "is:issue is:closed").
Note: some issues may remain open for years due to their evolving natures and complexities. When searching for open or closed issues, please examine all search results going back years. You can also sort your search results by "Recently updated" to find old issues that are still being worked on.


Maintenance metadata section

It's going to be stored in the EML, but maybe we don't need to display that on UI.

Moved updateFrequency to a dedicated sub-section in the basic metadata. Moved maintenanceDescription from the additional metadata


No proper visualizations to understand the dataset better

Prerequisites

[X] I am an Open Source Contributor
[X] I checked to make sure that this issue has not already been filed

Description
There are no proper visualization for a beginner to understand the dataset they will be working on for preprocessing the data.
Screenshots
No response
Code of Conduct

[X] I follow Contributing Guidelines of this project.

I can work on this sir!

There is no need for this visualisation. You are assigned with issue number 80 so pls work on that @Nabanita29


How to reset the existing graph then reload the checkpoint?

I need some help to do the following

train the network for some epochs with a dataflow
stop the training
do some evaluation on the training set
calculate a new probability for each sample being use for training process
update the dataflow
resume the training
repeat.

For now,  did as follow:
for period in range(0, N):
    train_dataflow = Dataset(dir_path)
    config = TrainConfig(
        dataflow=train_dataflow,
        callbacks=[ModelSaver],
        model=Model(),
        steps_per_epoch=steps_per_epoch,
        max_epoch=nr_epoch
    )

    if period != 0:
        config.session_init = SaverRestore(model_path)

    SyncMultiGPUTrainer(config).train()

    model_path = '...' # path to the most recent checkpoint
    new_dist = '...' # some calculation to change probability of sampled being used
    train_dataflow.dist = new_dist

    tf.reset_default_graph()

However, I got error when reloading the checkpoint at the next period. I'm sure this is not right way to do, any idea how can I achieve that? This is the error. Thanks.
Traceback (most recent call last):
  File "trainer_miccai.py", line 296, in <module>
    train(args, n_gpus)
  File "trainer_miccai.py", line 255, in train
    SyncMultiGPUTrainer(config).train()
  File "/home/vqdang/tools/anaconda3/envs/tf-gpu/lib/python3.6/site-packages/tensorpack/trainv1/base.py", line 175, in train
    self.setup()
  File "/home/vqdang/tools/anaconda3/envs/tf-gpu/lib/python3.6/site-packages/tensorpack/trainv1/base.py", line 210, in setup
    self._config.session_init._setup_graph()
  File "/home/vqdang/tools/anaconda3/envs/tf-gpu/lib/python3.6/site-packages/tensorpack/tfutils/sessinit.py", line 112, in _setup_graph
    dic = self._get_restore_dict()
  File "/home/vqdang/tools/anaconda3/envs/tf-gpu/lib/python3.6/site-packages/tensorpack/tfutils/sessinit.py", line 161, in _get_restore_dict
    self._match_vars(f)
  File "/home/vqdang/tools/anaconda3/envs/tf-gpu/lib/python3.6/site-packages/tensorpack/tfutils/sessinit.py", line 128, in _match_vars
    reader, chkpt_vars = SaverRestore._read_checkpoint_vars(self.path)
  File "/home/vqdang/tools/anaconda3/envs/tf-gpu/lib/python3.6/site-packages/tensorpack/tfutils/sessinit.py", line 122, in _read_checkpoint_vars
    reader = tf.train.NewCheckpointReader(model_path)
  File "/home/vqdang/tools/anaconda3/envs/tf-gpu/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 110, in NewCheckpointReader
    return CheckpointReader(compat.as_bytes(filepattern), status)
  File "/home/vqdang/tools/anaconda3/envs/tf-gpu/lib/python3.6/contextlib.py", line 88, in __exit__
    next(self.gen)
  File "/home/vqdang/tools/anaconda3/envs/tf-gpu/lib/python3.6/site-packages/tensorflow/python/framework/errors_impl.py", line 466, in raise_exception_on_not_ok_status
    pywrap_tensorflow.TF_GetCode(status))
tensorflow.python.framework.errors_impl.NotFoundError: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for /media/vqdang/Data_2/dang/output/miccai/v1.3.0_b//model-8

The way you're using should work. The error is saying the model path is wrong. Any chance you may have removed the checkpoint by mistake?
Everything other than the training can be done in callbacks. You can write a callback which, when triggered, run the evaluation (or get previous evaluation results) and change train_dataflow.dist. You can have the DataFlow be a subclass of Callback as well. In RL the data always depend on the training, so this has been used in both RL examples, e.g. in DQN dataflow is a callback.

I solved the problem. Using '//' in the path broke the tensorflow loader, I don't know why and it seems silly. Here the examples:
Above: /media/vqdang/Data_2/dang/output/miccai/v1.3.0_b//model-8.index
Fixed : /media/vqdang/Data_2/dang/output/miccai/v1.3.0_b/model-8.index
About the dataflow, is the queue reset every epoch? I haven't thought too much about that but the dataflow-callback way as in https://github.com/ppwwyyxx/tensorpack/blob/2c129dede46b8eb76e3f8a137d80d6accd4b48ff/examples/DeepQNetwork/expreplay.py#L109
looks more elegant and I want to try that. However, I need to make sure that from the next period, every samples used for training must be generated from the new distribution.

Ah that's a problem. If you use queues, there will definitely be several datapoints from the old distribution after you made any changes to the dataflow.
To really avoid this you will need to 1. pause your dataflow (make it block) and 2. Eval the inputs with sess.run + timeout option, until it timeout so the queue is empty. Perhaps this can be made a builtin method of QueueInput.

@ppwwyyxx I forgot to ask, after finishing SyncMultiGPUTrainer(config).train(), the existing dataflow will stop and existing queue will be removed right? Sorry to bring this up after closing the issue.

The dataflow will be stopped. The queue will be closed but not removed until you reset the graph.


Bump main to 6.0.0~pre1

Branch ign-rendering5 has been created for the Edifice release and 5.0.0 released from that.
The main branch now corresponds to version 6.0.0~pre1, and nightlies will soon be created from this branch.
https://github.com/ignition-tooling/release-tools/issues/421

No unexpected failures, this is good to go!


Fix styleguide missing message template

Fixes #1508

Codecov Report

Merging #1517 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1517   +/-   ##
=======================================
  Coverage   82.57%   82.57%
=======================================
  Files         131      131
  Lines        5665     5665
  Branches      643      643
=======================================
  Hits         4678     4678
  Misses        823      823
  Partials      164      164

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b01fd3b...66966b7. Read the comment docs.


Can chokidar handle diffing of files?

Looked around docs and searched issues to see if there was any inquiries for getting the diff changes of a watched file but couldn't find anything
I want to write a logging agent to send diff'd updates of a log file to a logging server so that the two copies can sync, logging server gets updates in real-time, and we don't bottleneck on filesize when things get large. With fs.watchFile, I can grab the previous and current file descriptor and compare bytes.
Is this possible in chokidar?

With fs.watchFile, I can grab the previous and current file descriptor and compare bytes.

You're comparing the stat? Or the actual contents of the file?
Chokidar doesn't read the contents of files in order to watch them - that's what would be needed in order to do before/after diffs. It is possible, but should be handled outside of chokidar.

@es128: ah, I've only done some research, but it's the fs.Stats class I will be working with so will not be comparing contents.

Chokidar can provide you with the fs.Stats with each event. It already does it when the data is already available via normal processing, or you can force it to happen with every event with the alwaysState: true option.
Combined with the default ignoreInitial: false, which will send you add events for all paths being watched upon instantiation along with file stats, you can hold those stat objects and diff them against new ones received with subsequent events.

Yep, I had to track the stats myself in a normalized dictionary. Was hoping for some convenience like what we get with watch, but it's working now.

@pthieu can you elaborate more on how you managed to get the diff?

Not near my code so can't share it.
But a library like watch implicitly holds the previous fs.stat data of the file. You compare the previous data with the new data to find the delta and parse the delta content into a buffer.
chokidar does not not keep the results of the previous fs.stat state so you have to do it yourself using whatever software pattern you like best.

@cvalenzuela: Not near my code so can't share it.
But a library like watch implicitly holds the previous fs.stat data of the file. You compare the previous data with the new data to find the delta and parse the delta content into a buffer.
chokidar does not not keep the results of the previous fs.stat state so you have to do it yourself using whatever software pattern you like best.

@pthieu, thanks!


Error when trying to use useState

Current Behavior
Without making any changes besides adding a useState to the component <Thing /> in the src file I'm getting this error:
TypeError: Cannot read properties of null (reading 'useState')
I'm not getting any errors in the DTS app console or my app console, it is compiling without errors or warnings. The error is showing when the component tries to mount. If I remove the useState my app is working fine.
This is how the code of the component looks like:

And this is the error I'm getting:

Expected behavior

Suggested solution(s)

Additional context

Your environment

Seems like it's happening with every react hook even though I'm using the react template.

I was not able to repro. what's your react/react-dom version? maybe using a version that does not support hooks?

I'm using 18.2.0

emm....can you make a repro?

ah. I got it! the cause is npm link. as said in the react docs:

This problem can also come up when you use npm link or an equivalent. In that case, your bundler might “see” two Reacts — one in application folder and one in your library folder. Assuming myapp and mylib are sibling folders, one possible fix is to run npm link ../myapp/node_modules/react from mylib. This should make the library use the application’s React copy.

https://reactjs.org/warnings/invalid-hook-call-warning.html#duplicate-react

Thanks @aladdin-add  that really helped 🙌🏻


Support for 32X16 and 1/8 scan RGB Modules

Hello NeoCat,
Can I use your library for 32X16 RGB (HUB75) modules. 64X32 modules's scan rate is 1/16, but the 32X16 RGB's 1/8 scan rate. I don't want to use other libraries for other modules.
And I also made HUB75 Shield for ESP32 suitable for your library. (if you provide your address I can ship sample to you.)
Thank you.

Hello !
Did you manage to use this library with a 32x16 display ?

Hello George,
No, I did not use this library for 32X16 panels, I used PxMatrix.

Hello George,
No, I did not use this library for 32X16 panels, I used PxMatrix.

You mean this one ?
https://github.com/2dom/PxMatrix

Hello George,
No, I did not use this library for 32X16 panels, I used PxMatrix.

You mean this one ?
https://github.com/2dom/PxMatrix

Yes, that works with most of panels with ESP8266 and ESP32.
NeoCat's library useful because of you dont need jumper cables end of the panels to input. But does not compatible except P3, P4 and P5 panels.

Hello George,
No, I did not use this library for 32X16 panels, I used PxMatrix.

You mean this one ?
https://github.com/2dom/PxMatrix

Yes, that works with most of panels with ESP8266 and ESP32.
NeoCat's library useful because of you dont need jumper cables end of the panels to input. But does not compatible except P3, P4 and P5 panels.

I have a 2727 SMD P5 64x32. It is a very strange module because it's P5 64x32 but it has a 1/8 scan pattern and I wasn't able to make it work with this library.
I've managed to use PxMatrix. But the issue with that one is that it accesses RAM to update the display (like most LED libraries do) and I'm trying to use me-no-dev's ESPAsyncWebServer library to serve some webpages in AP and STA mode.
Separately, both libraries work great, but when I put them together I get a lot of Guru Meditation Errors.

Hello GeorgeFlorian, Have you found the resolution for P5 64x32 with 1/8 scan pattern yet ?


Cow: Watch wptravel.io

Cow requests the watch of the watch_keyword wptravel\.io. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

wptravel\.io has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Rejected by cigien in Charcoal HQ. Reason: 'by request of the watcher'


Move google-api-services-storagetransfer to the new com.google.cloud library

Current api jar is:
com.google.apis:google-api-services-storagetransfer
In order to manage GCP "JOBS" in a better way, moving to some mutual interface of jobs (such as in BigQuery and com.google.cloud) can be very helpful.

Cloud Storage Transfer Service is essentially a separate service and is not part of our current Storage Client.
I agree that the storage and storage transfer clients are tightly related and it would be nice to have them in same place, but as of now (mostly for historical reasons) they are two very different things and merging them in a proper manner would require significant amount of work (just putting jars together is not an option here).

We are handling requests for supporting additional APIs with an internal mechanism. Closing out this issue.


🏗 Prevent shows without posters from being added to carousel

Now that the carousel shows poster cards, we should remove any shows that do not have a poster. We need to do this before we pass data to the carousel children so there are not gaps in each slide.

Closed by #485


Feature request: move clear button so it is centered with text

This library would be perfect if the clear button was centered vertically with the text/placeholder instead of the entire view.

I went in to all the CGRect() ctor's and subtracted 6 from the y. Seems to make everything visually align then with the clear button. Total hack, but did the trick.


Error found when preparing "var.tile.mtx" in the tutorial

> var.tile.mtx <- tile.mtx[match(var.tiles, tile.mtx@elementMetadata), match(archr.obj$cellNames, colnames(tile.mtx))] Error in h(simpleError(msg, call)) :    error in evaluating the argument 'i' in selecting a method for function '[': error in evaluating the argument 'table' in selecting a method for function 'match': no slot of name "elementMetadata" for this object of class "dgCMatrix"

I am also having this same issue. It appears to be an ArchR issue -- for some reason the tile.mtx@elementMetadata is NULL when it should return the tile coordinates.

Using the code in addTileMatrix() and .addTileMat() from MatrixTiles.R in the ArchR source code, you can reconstruct the dataframe storing metadata for all genomic tiles as follows:
input <- archr.obj
ArrowFiles <- getArrowFiles(input)
chromSizes = getChromSizes(input)
blacklist = getBlacklist(input)
chromLengths <- end(chromSizes)
names(chromLengths) <- paste0(seqnames(chromSizes))
blacklist <- split(blacklist, seqnames(blacklist))
excludeChr = c("chrM", "chrY", "chrX")     # adjust according to your preferences
chromLengths <- chromLengths[names(chromLengths) %ni% excludeChr]
tileSize <- 500
featureDF <- lapply(seq_along(chromLengths), function(x){   DataFrame(seqnames = names(chromLengths)[x], idx = seq_len(trunc(chromLengths[x])/tileSize + 1)) }) %>% Reduce("rbind", .)
featureDF$start <- (featureDF$idx - 1) * tileSize
Then you can filter for variable tiles by substituting featureDF for tile.mtx@elementMetadata:
var.tile.mtx <- tile.mtx[match(var.tiles, featureDF), match(archr.obj$cellNames, colnames(tile.mtx))]
This probably isn't the cleanest way to get around this issue but it worked for me!

Hi @jcurrie7, your code worked for me. Thanks for your help!

Hi @jcurrie7 , were you able to get the downstream CellSpace command line to work? Mine is seg faulting.
This is my output:
CellSpace -output data/CellSpace_embedding-var_tiles -cpMat  data/CellSpace_cell_by_tile-counts.mtx -peaks data/CellSpace_var_tiles.fa  CellSpace Arguments: dim: 30 ngrams: 3 k: 8 sampleLen: 150 exmpPerPeak: 20 epoch: 50 margin: 0.05 bucket: 2000000 label: '__label__' lr: 0.01 maxTrainTime: 8640000 negSearchLimit: 50 maxNegSamples: 10 p: 0.5 initRandSd: 0.001 batchSize: 5 saveIntermediates: 'final' thread: 10. Start to initialize starspace model. Number of words (8-mers) in dictionary:  32896 Number of labels in dictionary: 77141 Reading 9964 peak sequences from 'data/CellSpace_var_tiles.fa' Segmentation fault (core dumped)


Yeoman Generator

Hello,
do you plan on making a Yo generator ?
Thanks

I haven't created a yo generator before so I probably may not be able to contribute this. I you have experience with this, a PR would be welcome!


fix jenkins script for chrome extension + updating legal files for 10.0.0 rc4

backport files for 10.0.0 rc4 release + Jenkins script fix


added config for dynamic snippet mapping

merging this requires merging branch feature/video-manager on sulu/sulu

merged in d1db4ba


[Guides] Fix dead link in migration.md

I actually have these all fixed in a new local commit for #1025, pushing it now

👍


Axis Rules: limit rules do not function property on inverted axes

@VisMotrix recently added support to the square axis rules in #3592, but the inner/outer boundary rules (demonstrated by the demo application) do not function as expected with inverted axes. I added "inverted axis" checkboxes to the demo app to demonstrate the isssue:

In MaximumBoundary.cs, when the axes are inverted, the condition seems to be invalid.

After further investigation, in FractionalAutoScaler.cs, when the axes are inverted, the maximum and minimum values are swapped.
This might be causing the issue.


Fix Admin: UserController and Flight Edit

UserController: Use Role Repository and selectBoxList method instead of the model itself.
Flight Fields: Add empty option to Days select field

Closes #939
Closes #938
For both ; If users wants to clear out that fields, all options must be removed and empty/blank option must be selected before updating the record. Deleting only the days or roles without selecting the empty/blank option will not be enough to clear out that fields.

Thank you sir, yep, one of those cleanup things and glad it fixed that. This was some of the first code written


Details row template with custome javascript

I am working on an application where i want to show some sensor readings in real time. I have a CRUD for devices and in the details row of each entry for a device i want to show a realtime javascript chart which will show the readings coming from that device, which requires some js to be added along with the template for details row.
I tried creating a new blade file for the details row and added the js at the end of the body tag but it did'nt work.
I also tried adding adding the js files in master layout file but it didnt work as well, however it did work when i loaded the chart inside dashboard.blade.php, but not working inside details row.
Any help would be appreciated. Thanx

Hi @badarkhan1 ,
I'm afraid it isn't possible to loads JS in the table cells. They're loaded using an AJAX call, which returns the actual HTML. So even if you did have javascript inside the cell, it wouldn't be run and it would not affect the DOM, it would just be placed inside the HTML by DataTables.
I recommend you create a custom column type without javascript, but with the indicators you need, something like:
id }}" class="custom-graph">{{ str_limit(strip_tags($entry->{$column['name']}), 80, "[...]") }}

and things with Javascript separately. In the example above you'd be able to use the $(".custom-graph") selector and then the entry-id data attribute. You can use custom JS inside the list view using one of two methods:

by loading a custom blade file with $this->crud->setListView('your-view');
by modifying public/vendor/backpack/crud/js/list.js

Hope it helps.
Cheers!

Hey.
I had to show a khill/lavacharts in my detail row.
And i had to return JS in ajax response.
I ran into a similar problem.
And i made this solution
In MyCrudController.php
public function showDetailsRow($id)
{
...
return [
'html'  =>view('vendor.backpack.crud.details_row')->render(),
'js' => $this->buildDataTable()->render('CalendarChart', 'Calendar', 'callendar-container-'.$id),
}
Then i modified the details_row_logic.blade.php in ajax.done method
row.child("" + data.hasOwnProperty('html')?data.html:data + "", 'no-padding').show();
if(data.hasOwnProperty('lava')){
$('body').append( $(data.lava) );
}
I hope this will help for someone

I am working on an application where i want to show some sensor readings in real time. I have a CRUD for devices and in the details row of each entry for a device i want to show a realtime javascript chart which will show the readings coming from that device, which requires some js to be added along with the template for details row.
I tried creating a new blade file for the details row and added the js at the end of the body tag but it did'nt work.
I also tried adding adding the js files in master layout file but it didnt work as well, however it did work when i loaded the chart inside dashboard.blade.php, but not working inside details row.
Any help would be appreciated. Thanx

Hey.
I had to show a khill/lavacharts in my detail row.
And i had to return JS in ajax response.
I ran into a similar problem.
And i made this solution
In MyCrudController.php
public function showDetailsRow($id)
{
...
return [
'html'  =>view('vendor.backpack.crud.details_row')->render(),
'js' => $this->buildDataTable()->render('CalendarChart', 'Calendar', 'callendar-container-'.$id),
}
Then i modified the details_row_logic.blade.php in ajax.done method
row.child("" + data.hasOwnProperty('html')?data.html:data + "", 'no-padding').show();
if(data.hasOwnProperty('lava')){
$('body').append( $(data.lava) );
}
I hope this will help for someone

Thanks for sharing your solution @censore ! I’m sure it’ll help someone :-)


Edit hdinsight-connect-excel-hive-ODBC-driver.md

Edit complete.
The placeholder "" on line 46 isn't rendering correctly in the GitHub preview.

Hi @ShawnJackson, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (v-shawja). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Demo needed

Still quite confused about the final implement of RSS3,  a runable demo should be helpful~

Still quite confused about the final implement of RSS3, a runable demo should be helpful~

If this is not possible for current stage, some user stories in real life can also help

This is a simple example for publishing content on an RSS3 endpoint network.
import hexbytes
import requests

from eth_keys import keys

if __name__ == "__main__":
    endpoint = "https://rss3-hub-playground-6raed.ondigitalocean.app"
    pk = "0x47e18d6c386898b424025cd9db446f779ef24ad33a26c499c87bb3d9372540ba"
    pk = keys.PrivateKey(hexbytes.HexBytes(pk))
    address = pk.public_key.to_checksum_address()
    print(address)
    r = requests.get(f"{endpoint}/{address}")
    print(r.json())

This is a simple example of fetching content from the RSS3 endpoint network.
import hexbytes
import requests

from eth_keys import keys

if __name__ == "__main__":
    endpoint = "https://rss3-hub-playground-6raed.ondigitalocean.app"
    pk = "0x47e18d6c386898b424025cd9db446f779ef24ad33a26c499c87bb3d9372540ba"
    pk = keys.PrivateKey(hexbytes.HexBytes(pk))
    address = pk.public_key.to_checksum_address()
    print(address)
    r = requests.get(f"{endpoint}/{address}")
    print(r.json())

pretty cool,  seems ppl using public_key as their id to publish content, and private one to denote the ownership ? Any node api provided?

Yeah, you can try it through RSS3-Hub RSS3-SDK-for-JavaScript RSS3-Python-SDK and some upcoming libraries


https://radar.cloudflare.com/scan/e820f36a-f3a1-4733-b902-40677aa15697/summary

…7/summary

This PR is:

[x] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the Cloudflare report
[x] This site is not an ultra minimal site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: architjain-512.pages.dev
  url: https://architjain-512.pages.dev/
  size: 430.7
  last_checked: 2024-08-12

Cloudflare Report: https://radar.cloudflare.com/scan/e820f36a-f3a1-4733-b902-40677aa15697/summary

Thanks!


(Draft) SQHELM-116 install_plugins.sh: fix plugins deletion

PR Draft needed to overcome the external PR issue
The install_plugins.sh script starts by deleting previously installed plugins, but fails in case there are several already installed. As a result, updating a plugin leads to SonarQube not starting (because having two versions of the same plugin is a fatal error).
Fix this by using rm -f * instead of broken [ -e * ] && rm *.
Please ensure your pull request adheres to the following guidelines:

[ ] explain your motives to contribute this change: what problem you are trying to fix, what improvement you are trying to make
[ ] Document your Changes in the CHANGELOG.md file of the respected chart as well as the Chart.yaml
[ ] Bump the Version number of the respected chart

Drafter PR only to trigger the CI on the original coming from forked repos


reverts app_url changes to fix system tests

the system tests are actually failing for a different reason so closing this


correct 'you problem' to 'your problem'

department of a 'not a native speaker here' but either that grammar is wrong as I suspect, or the joke is lost on me.  which happens ...

Thanks, I don't see it deployed yet.  Will check again in a few days.


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In 329d54a, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 9301 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in 8b76f2b after 26 minutes.


What are the current FD product(s) and how do they fit together?

When I implement improvement measures on my website, I want to know what's the current state of products and features as well as the connection between them, so that I can answer questions like "Do we need new products?", "How do our products fit together?" etc...
Acceptance criteria

[ ] List of the products in the FD Products spreadsheet table
[ ] Every item has a description
[ ] Diagram of their relation
[ ] Gap analysis on current products / subproducts

Tasks

[ ] Do a quick research on the FD products
[ ] List them in the spreadsheet
[ ] Add descriptions and other basic info
[ ] Create a diagram illustrating their relation
[ ] A gap analysis on current product / subproducts

@cabral i think this is a bit tough for @dipeshkhadkaa to do at this stage.
I think this needs someone with a good grasp of the project e.g. me 😉 - i can do this with someone so they learn.
/cc @johanricher @cbenz do you want to get involved?

@cabral I am trying to add as many products as I can including the tools from the website.
Also, I have posted my questions as a hypothesis for which rufus has answered in above comment.

In the sheet named "Products," you will find a list of all the products I could identify from the list of websites and repositories. It contains product names, clusters (type of product and purpose), major links, official descriptions, suggestions of better descriptions, and comments. The comments should help to address the state of each product – a definitive state will require more time for testing each of the tools.
I'm moving the task of creating the diagram of products to a specific issue and closing this one. The information present in the previously mentioned sheet should be sufficient to create such diagram. Also, it gives a great understanding of what Frictionless Data means in practice nowadays. I'm eager to see it more visually.


Dialog cannot be opened from Drawer

Whenever I try to open Dialog from Drawer, the Dialog shows up for very short time ( til the drawer is closed entirely ) and disappears.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Dialog shows up while the Drawer slides back in the background.
Context

I am react newbie but I am pretty sure that is problem caused by this framework.
I do not know how these components works, but from the behavior I guess when the Drawer stops rendering, the Dialog stops aswell as it is directly in it.
Your Environment

Tech
Version

Material-UI
1.0.0-beta.8

React
15.6.1

browser
Chrome, v60

Your issue has been closed because it does not conform to our issue requirements.
Please provide a reproduction test case. This would help a lot 👷 .
A live example would be perfect. This codesandbox.io template may be a good starting point. Thank you!

I added those requirements and live example aswell. Can you re-open it or should I create a new issue ?

This behavior is expected. The content of the Drawer is only rendered when open. Hence, you Dialog is getting dismissed. You need to have some state higher in the React rendering tree.


Feat: Edit the Description

Pretty straightforward, I'm planning to introduce the ability to edit the description for an MR from within the editor. Right now you can view a summary with :lua require("gitlab").summary() but you cannot edit it.
I am willing to submit a PR for this change

[x] yes
[ ] no

If you pull down, does that fix things? I wasn't using the pre-initialized client, so the base URL was wrong.

yep, perfect!


Publish list of reserved property names

Using a property name that Polymer uses can lead to some very subtle bugs. We should publish a list of reserved public and private properties names that elements should not use. The linter should warn on usage.

Just spent an hour trying to debug what came to be the [unadverted] usage of a reserved property name. This will be definitely a helpful documentation.

+1

+1
Spent better part of a day debugging something that turned out to be a reserved word 'profile'

Faced this issue for 1.9 today by defining property with name _serialized. This stopped working attribute bindings for element.


[WIP] add (optional) support for Elasticsearch

This PR adds optional support for Elasticsearch and amends docker-compose.yml to show how it can be used. It's also fairly straight forward to configure on AWS.
Feedback welcome.

This looks solid to me :1st_place_medal:


Explore button filters & stack navigation

There are a number of entry-points to Explore, each with different contexts and expectations about what view is presented and what the filters are. Furthermore, most of these buttons push Explore onto an existing stack navigator, e.g. MyObs -> ObsDetails -> TaxonDetails -> Explore, or even more confusingly, Explore -> TaxonDetails -> Explore, and in these stacked contexts, Explore needs a back button to pop it off the stack and return to the previous screen (plus the previous Explores need to keep their state).

[x] MyObs toolbar Explore button should

[x] Show the Observations view
[x] Show a back button that takes you back to MyObs
[x] Have an accessibility label that makes it clear you are going to see all of your observations in Explore
[x] Filter observations by the current user with no other filters, i.e. user_id=1&verifiable=any and no default geographic filters like nearby

[x] TaxonDetails Explore button should

[x] Show the Observations view
[x] Have an accessibility label that makes it clear you are going to see observations of this taxon in Explore
[x] Show a back button that takes you back to TaxonDetails
[x] Filter observations by the taxon that are verifiable with no other filters, i.e. taxon_id=1&verifiable=any and no default geographic filters like nearby

[x] UserProfile observations button

[x] Show the Observations view
[x] Show a back button that takes you back to UserProfile
[x] Have an accessibility label that makes it clear you are going to see observations by that user in Explore
[x] Filter observations by the profile user that are verifiable with no other filters, i.e. user_id=1&verifiable=true and no default geographic filters

[x] UserProfile species button

[x] Show the Species view
[x] Show a back button that takes you back to UserProfile
[x] Have an accessibility label that makes it clear you are going to see species observed by that user in Explore
[x] Filter observations by the profile user that are verifiable with no other filters, i.e. user_id=1&verifiable=true and no default geographic filters

[x] Bottom tabs Explore button should

[x] Default to species view (#1231)
[x] Default to Nearby with location permission otherwise Worldwide geographic filter
[x] Not lose its state when you navigate away, i.e. when you are in Explore and set some filters then tap Notifications in the bottom tabs and then tap Explore in bottom tabs, those filters and those observations should still be there  (#1215)
[x] Not have a back button

I would try to experiment with at least two different screens, one for the root Explore accessed from the bottom tabs that should never have a back button, and another for the Explore that can be stacked (with a back button). Ideally every stacked screen is a separate instance and doesn't navigate you back when you do something like Explore -> TaxonDetails -> Explore -> TaxonDetails -> Explore. Alternatively, maybe it would work if the bottom tabs button just resets the nav state.

@kueda there's one other entry point I see for explore, from the logged out MyObservations screen. should that one operate similar to the bottom tabs entry point -- species view, default to nearby if location permissions are given, and no filters except verifiable? and should it have a back button to return to MyObservationsEmpty?

Let's treat it exactly like the bottom tabs entry point, no back button.

closed by PR https://github.com/inaturalist/iNaturalistReactNative/pull/1306


Incompatible with compiler version 0.8.20

Description
Mythril will always alert that there is an exception state at PC address 12 if we compile contracts with compiler version 0.8.20.
How to Reproduce
We will illustrate the issue using the following sample contract:
// SPDX-License-Identifier: GPL-3.0
pragma solidity 0.8.20;

contract SimpleStorage {
    uint storedData;

    function set(uint x) public {
        storedData = x;
    }

    function get() public view returns (uint) {
        return storedData;
    }
}

When we use Mythril to analyze the contract, it will produce the following outputs:
$ myth analyze SimpleStorage.sol
==== Exception State ====
SWC ID: 110
Severity: Low
Contract: SimpleStorage
Function name: constructor
PC address: 12
Estimated Gas Usage: 39 - 134
An assertion violation was triggered.
It is possible to trigger an assertion violation. Note that Solidity assert() statements should only be used to check invariants. Review the transaction trace generated for this issue and either make sure your program logic is correct, or use require() instead of assert() if your goal is to constrain user inputs or enforce preconditions. Remember to validate inputs from both callers (for instance, via passed arguments) and callees (for instance, via return values). This issue is reported for internal compiler generated code.
--------------------
Initial State:

Account: [CREATOR], balance: 0x2, nonce:0, storage:{}
Account: [ATTACKER], balance: 0x0, nonce:0, storage:{}

Transaction Sequence:

Caller: [CREATOR], calldata: , decoded_data: , value: 0x1

Mythril indicates that there is an exception state at PC address 12 if the callvalue is not zero.
However, if we change the version pragma to pragma solidity 0.8.19;, i.e., to compile the contract with compiler version 0.8.19, Mythril will alert no issues:
$ myth analyze SimpleStorage.sol
The analysis was completed successfully. No issues were detected.

I guess there may be some incompatible changes between the bytecode generated by compiler version 0.8.20 and the others which checks the zeroness of the callvalue.

Hi, thanks for pointing it out, it should be fixed in the latest commit


Allowing credentials that don't implement create_scoped().

@tseaver

Maybe this isn't the best way to solve #1412? Happy to hear suggestions.
I also realized that we don't really need the get_for_service_account_p12 and get_for_service_account_json helpers any more since the ServiceAccountCredentials factories do everything in a nice short way.

LGTM


ERROR

GOOGLE API ERROR
Daily Limit for Unauthenticated Use Exceeded. Continued use requires signup.

If you refresh (maybe with a cache clear) you should find the problem resolved.


[client][Retrieve Token] Issues passing creds with Content Type "application/x-www-form-urlencoded"

Hi there,
Using the api client, I have an issue when retrieving token ( in the frame of the installation of the newest test tower in Ardeche)
With correct creds (tested at https://api.pyronear.org/docs ), it basically works with some creds, and not with others !
At first, I thought it might be due to the encoding of the characters, as "application/x-www-form-urlencoded" as Content type
is not recommended to send text with non ASCII characters, in the request headers.
Hence, switching from request with application/x-www-form-urlencoded to a "classic" post request with a json data, solves the issue.
However I realise that this solution is fortuitous because creds tested do not contain non-ASCII characters 🤣
So, to be pragmatic, why using content type application/x-www-form-urlencoded in the first place to retrieve token ?
Is there any specific reason ? Otherwise, this PR could be useful to avoid the problem mentioned above
happy to discuss it !
If anyone has an explanation as to why this doesn't work, I'm interested (text encoding then decoding causing issues?)

@pyronear/back-end

@MateoLostanlen @Akilditu

Oh now I understand: when I generate the creds for this, there were indeed perhaps rarer characters. So the encoding might be different from the swagger :+1:
Let me take a look at the PR!


Pasting does not fire tbwchange

Informations
It seems that when you paste text into the editor, the tbwchange event is not fired at all unless you have the removeformatPasted option set to true. In the issue #579 it was reported that the tbwchange event was not fired for IE 11 when this flag is set to true, and the fix was to simply add the trigger but this does not resolve this issue.
I have been able to fix it on our local copy of trumbowyg by changing the code at line 690 from
setTimeout(function () {
    t.semanticCode(false, true);
    t.$c.trigger('tbwpaste', e);
}, 0);

to
setTimeout(function () {
    t.semanticCode(false, true);
    t.$c.trigger('tbwpaste', e);
    t.$c.trigger('tbwchange', e);
}, 0);

And removing the t.$c.trigger('tbwchange', e); on line 679 as this would no longer be necessary.
I am unsure whether this has any further implications to other things but this resolves the issue.
I've tested this on Chrome for Windows and Mac, both of which showed the issue. The only way to make the value actually change was to leave the input field, then give it focus again.

I am too facing the issue and waiting for solution.

The simple workaround is to hook your code up to the tbwpaste event as well as tbwchange (and possibly tbwblur too if you want to be fully belt and braces).

Facing the same problem. Even the tbwpaste event isn't fired for me when you paste something.

@geoffbeaumont I tried hooking 2 events like this,
$(DiagnoseTextBox).trumbowyg().on({                 'tbwpaste': function () {                     textCounter(DiagnoseTextBox, CharactersRemainingLabel, 4000)                 },                  'tbwchange': function () {                     textCounter(DiagnoseTextBox, CharactersRemainingLabel, 4000)                 }             })
It doesn't seem to work. Could you please explain how you fixed it?

@pareeohnos I tried your workaround in my local copy, but it doesn't seem to work.

@MirazMac I'm pretty sure tbwpaste did work in our environment. I can't remember whether I implemented onblur or updated the underlying form field from a form submit event (we have trumbowyg in a form which is submitted by clicking a button), but we've had no problems since.

@geoffbeaumont Well, no luck for me. I hope this would be fixed soon.

Make me a MR with the fix and I will merge it ASAP :)

Done in next version which will be deployed soon.


Avoid constants with side effects

Relates to https://github.com/api3dao/airnode/issues/217

I went through all of the named constants in the codebase. The problem is that we have defined generic named constants (e.g. RANDOM_BACKOFF_MIN_MS, DEFAULT_RETRY_TIMEOUT_MS) but these could be used in different places and not necessarily bound to be the same. I renamed these generic names to a better name (more specific one).
I also removed some outdated and unused constants.
Currently, we define the API call timeout to 30s and processing timeouts to 5s each. We define the worker timeout also to 30s. However, API call + processing can last up to 40s, so it felt tempting to change these timeouts to gracefully account for this worst case. I was thinking about reducing the API call timeout to reserve time for processing is too limiting. It's better to let it timeout by the worker timeout.
It's also possible that the TF timeout will kill the running lambda, because it feels hard to guarantee what is the total execution of a handler, because blockchain calls are retried once when they fail. I think this is OK though, but I am not sure if our pending logs feature will handle this correctly (e.g. TF kills the lambda before logger.logPending is called and we end up with no logs).

I guess you shouldn't merge this to master as it's under the 0.10 milestone.

I guess you shouldn't merge this to master as it's under the 0.10 milestone.

I've created temp-v0.10.0 branch to merge this into.

Is the primary reason for holding this back risk mitigation?
(and this is a welcome change)

Is the primary reason for holding this back risk mitigation (too many changes at once)?

I think the primary reason should be to honor the things we decide and plan. It is a bit unfortunate that plan for 0.9 changed, but since we decided this should land in 0.10 it should not be released sooner.


internet disconnected of LAN when the simulation runs

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04):18.04
Apollo installed from (source):source
Apollo version (3.0 and 3.5):3.0 and 3.5,

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:
Hi All:

I followed the steps to build and launch demo, watch it via 8888, and everything works fine
the internet was disconnected, and everyone in our LAN (more than 30 persons) was offline during the period when I ran the simulation, whatever big loop or others.
I didn't have hardware, running the simulation only
when I tried it at my home ( only 2 persons in a LAN), everything functioned well, without disconnection issue.

Supporting materials (screeshots, command lines, code/script snippets):

@gvc0461082002 we have never seen this happen? Did it happen just that once?

@gvc0461082002 we have never seen this happen? Did it happen just that once?

HI Natasha:
The problems happened since ubutnu16.04/apollo3.0
Actually, the bandwidth cost ranges from 20k to 40k , as displayed from system monitor.
I did use lsof-i to get all links details.
When I stopped the docker, usually there are more than 8 containers will be closed.
I am not sure the issues from communications among nodes.
Another finding is map loading of Navigation is fast at home while it's slow in the company's LAN.

@gvc0461082002 I am not sure why this is happening on your end. It is not something that we face internally or something I have faced on an external network. Are you sure this is occurring due to Apollo and is not an external network issue?

Hi Natasha:
It could be a network issue not related to  ApolloAuto.
It hasn't been figured out by me now. But I like to this one closed.
The uploading rate rang from 30-50kib/s indicated by ksysguard under kde18.04, incuring
the jamming issue.
Cheers
Fenglu Ge
------------------ Original ------------------
From:  "Natasha<EMAIL_ADDRESS>Send time: Thursday, May 23, 2019 8:46 AM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject:  Re: [ApolloAuto/apollo] internet disconnected of LAN when thesimulation runs (#8374)
@gvc0461082002 I am not sure why this is happening on your end. It is not something that we face internally or something I have faced on an external network. Are you sure this is occurring due to Apollo and is not an external network issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@gvc0461082002 sure I will close the issue. If you have any further questions, feel free to reopen it. Thanks!


Could not cast value of type '__NSCFNumber' (0x10881b608) to 'NSString' (0x1f473fc40).

Describe the bug
Adding a number to http param produces the following error:
fieldId 2140
Could not cast value of type '__NSCFNumber' (0x107bf3608) to 'NSString' (0x1f473fc40).
2022-05-03 09:09:54.836692+0200 App[7297:240873] Could not cast value of type '__NSCFNumber' (0x107bf3608) to 'NSString' (0x1f473fc40).
dyld4 config: DYLD_LIBRARY_PATH=/usr/lib/system/introspection DYLD_INSERT_LIBRARIES=/Developer/usr/lib/libBacktraceRecording.dylib:/Developer/usr/lib/libMainThreadChecker.dylib:/Developer/Library/PrivateFrameworks/DTDDISupport.framework/libViewDebuggerSupport.dylib
(lldb)

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Numbers should correctly cast to NSString.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Totally my bad. Our project has no strict typing and I wasn't checking what type Params is expecting. Will cast it to String in JavaScript first.


Restore content-type auto-detection and only optionally disable it

Request Statement
Hi,
in release 5.0.0 the content-type auto-detection logic. This seriously impact many applications relying on that logic (which, for many use-cases, was correct). SInce it was already possible to force manually content-type, would'nt be possible to specify a constructor flag in Dio to trigger the new logic and let the old one works flawlessly for compatibility with all existing code ?
Thank you,
Solution Brainstorm
Pass parameter on constructor to explicitly disable the content-type auto-guess logic.

I understand your point of view, but I hope you have considered all
implications of this breaking-change since it will surely break many
apps; as an example, we built an app with 200+ api calls mostly relying
on auto-detection logic and we now have to review all of them since the
API server needs to know which kind of content is being sent to it. In
serious app (not the example ones where Dio instance is unique and
shared across all functions) Dio can be used in multiple locations and
it needs to talk to external hardware with embedded web-servers. Some of
them, for example, simply crash (requiring hardware reset) if requests
are sent with wrong content-type.
I suggest too to add a flag to let preserve header variable
case-sensitivity for the same reason; dumb devices (whose firmware is
immutable) don't understand lower-case only headers.
Thank you for your work.
Il 17/02/2023 18:05, Alex Li ha scritto:

An exposed API would be considered as used by everyone. The
improvement we might be able to do is to provide an auto-implying
interceptor, but that's quite equivalent to manually setting the
|content-type|.
Closing as won't fix.
—
Reply to this email directly, view it on GitHub
https://github.com/cfug/dio/issues/1667#issuecomment-1434936017, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANTEOJNRBTXI5S55QVW7JTTWX6VWBANCNFSM6AAAAAAU7OVRUA.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Ing. Luca Lazzeroni
Responsabile Ricerca e Sviluppo - Trend Servizi Srl
Tel. +39-0376-631761
https://www.trendservizi.it


Custom configuration for CORS headers

Summary
CORS headers cannot be customized. As a consequence, we can't ser the Access-Control-Allow-Credentials header, nor set extra headers for Access-Control-Allow-Headers.
Steps to reproduce
Case 1: Mock any valid schema, and make a CORS request with a custom header such as x-foo: bar
Case 2: Mock any valid schema, and make a CORS request using withCredentials (https://developer.mozilla.org/en-US/docs/Web/API/XMLHttpRequest/withCredentials)
Actual result
The request fails indicating some of this errors, depending on the case:
Case 1: [...] Request header field x-foo is not allowed by Access-Control-Allow-Headers in preflight response
Case 2: [...] Response to preflight request doesn't pass access control check: The value of the 'Access-Control-Allow-Origin' header in the response must not be the wildcard '*' when the request's credentials mode is 'include'. The credentials mode of requests initiated by the XMLHttpRequest is controlled by the withCredentials attribute
Expected result
The mock server should be able to:

Set/add allowed headers whether via configuration or API schema custom fields
Set CORS allow credentials property (in this case, it should set the header Access-Control-Allow-Credentials: true, and set the Access-Control-Allow-Origin with a dynamic value (with allow-credentials, origin cannot be a wildcard)

I've tested both cases with the latest code and can confirm they are now working as expected. Please let me know if you still want manual customization. For now the goal was to have it generally just work.


Could not stage for compilation: Could not find ref

When I am watching directory via
gosass --sourcemap --load-path scss  -input scss/ -output css/ --watch

and I change the sass file — I get the following error
Could not stage for compilation: Could not find ref `../_scss/foo.scss` (tried '{PATH_OMMITTED}/redesign/_scss/_foo.scss.scss' and '{PATH_OMMITTED}/_scss/foo.scss.css')

The import statement in scss is like
@import '../_scss/foo.scss';

It's strange that first compilation works, but later — it fails.

It should be @import '../_scss/foo.

I've added a pull request that fixes this issue — https://github.com/dailymuse/gosass/pull/11


Settings editor is awesome - could make it available to user projects

Settings UI and the backend json is awesome. It's simple and the paradigm is very generic. It could be used for managing config/settings of other projects. Eg. settings of Windows Terminal
This whole paradigm could be made generic, documented and UI could be made available to other projects.

It's not actually generic, it's tightly tied to vscode's settings model.

vscode's settings model - the backend and frontend - is fairly generic. Configuration and settings are a necessity for every application out there. But good UX for settings is a rarity. vscode solved this UX issue and I propose this solution could be made available to other apps.

Users could use vscode settings model (i.e. documented jsons) for their own projects
If some key of json is present (say like schema), vscode interprets these jsons and opens UI for editing these jsons
Taking 1 and 2 to extreme, settings model and UI could be separated out into a package which could be reused by other apps. I understand that this is more work.

I hope this makes my proposal clearer. I must admit I don't understand the internals of vscode very well - I'm a python dev. But I can't but appreciate our settings module and wish it is available for my project.

Thanks for the idea but what you are proposing isn't a feature request, it's like a new product request. The code is tightly coupled to vscode and there is a lot of it. It's not a generic json schema editor, it's optimized for vscode's extensions of this and the subset that we use.

Thanks for the idea but what you are proposing isn't a feature request, it's like a new product request

I agree!

Someone else is welcome to do the huge refactoring necessary to make this useful for other products but I think it might be faster to start over.

I think you're right about this. Starting over is easier. Would you be able to point me to the current code for settings editor and its backend. It'll help me and others whoever is interested in such a project. Thanks!

Many times, parts of Code are so good, but too tightly coupled with other parts to be extracted or reused.

You can find the file settingsEditor2.ts and start from there.

Thanks!


ci: configure setting up node.js and caching node modules

Type
enhancement

Description

Added Node.js setup to the CI workflow to ensure the use of a specific Node.js version (20.11.0).
Implemented caching for node modules to improve build times.

Changes walkthrough
Relevant filesEnhancement

ci.ymlEnhance CI Workflow with Node.js Setup and Caching

.github/workflows/ci.yml
Added steps to setup Node.js with a specific version (20.11.0).  Added caching for node modules using actions/cache.

+14/-0

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add generic Hashing endpoint to server and the corresponding Frontend functions.

Adds

[x] api changes that replace blake2bhash with plutusHash
[x] fix documentation for server endpoint
[x] corresponding frontend changes
[x] clean up everything, add the wrapper functions in Contract.Crypto
[x] add integration test
[ ] actually run the integration test??

Issues

Hi, thank you for your review. Can you tell me how I run the integration tests? Does it automatically pick them up if I have the #ctl-env thingy running? (Had some permission issues with that)

Can you tell me how I run the integration tests?

@MangoIV you need to:

execute nix run -L .#ctl-runtime
execute spago test -m Test.Integration


Adds UnstableTrait and UnstableTraitValidator

Adds UnstableTrait that can be used to indicated a shape is unstable and may change in the future.
Adds UnstableTraitValidator that emits warnings when a model contains shapes that apply unstable traits.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Are we only using this as a meta-trait, or can it be applied to any given shape? Your trait selector lets it be attached to anything, but you only emit meta-trait warnings

I think it makes sense to only look at traits that are unstable in the IDL. Warnings should be emitted, when possible, in programming languages when unstable shapes are used, but in the model, it makes sense for shapes to refer to unstable shapes. For example, I could have an unstable operation bound to my service because I want to beta test it. Emitting a warning for that on the service would be just noise IMO.

Can you also target 0.10 instead?


Delete Icon, so Windows users can contribute

Catch if one or more 'holders' sections are empty
Catch if 'sustainability' is empty
Check that 'regularMarketOpen' is available

I will second this request. I had to make my own fork and delete the Icon file just to install the package.

The Icon prevents me from cloning the repo. Please incorporate this PR.
Although the message with this PR doesn't make much sense.

Message on the PR does not match title or change.  However, I will merge now to remove the icon file.


Call custom function inside renderitem gives error

outrun(){
alert('aa')
this.props.navigation.navigate('ProductDetail', { name: uri.replace('/product/','') , token:this.state.token })
}

_renderItem ({item, index}) {
var self=this;
    return (
        <TouchableOpacity onPress={()=>this.outrun()} style={{height:180,backgroundColor:'#fbb034'}}>
	<Image style={{width:Dimensions.get('window').width,height:150}} resizeMode={'contain'}  source={{uri: 'https:'+item.Image}}/>
            <Text style={{fontSize:18,color:'white'}}>{ item.Description }</Text>

	</TouchableOpacity>
    );
}

⚠ You did not follow the contributing guidelines!
As stated in these:

You need to fill out the issue template. This step is mandatory! Not doing so will result in your issue getting closed. Don't take this personally if this happens, and feel free to open a new issue once you've gathered all the information required by the template.

Your issue looks like a classing binding mistake and has nothing to do with the plugin. See this comment for more info.
There is also an entire section of the doc dedicated to implementing navigation. Please read it!


feat(db): crete a simple mongoose demo

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Alternative to https://github.com/freeCodeCamp/freeCodeCamp/pull/49413
I'm not particularly familiar with Mongoose, so I could have missed some obvious ways to improve this. However, as is, it does not compare favourably to Prisma.  While both suffer from the fact that the schemas have to be defined by us, Prisma does better in two respects.
Firstly it's inferred schema seems accurate (if incomplete) and it highlights where the data type was ambigous. Secondly, and more importantly, the types that Prisma generates are accurate. The queries you can write are correctly constrained and the returned data has the expected types. Neither is true for Mongoose.

All good points, though I'd rather not clutter up the fastify instance with all the collections. How about fastify.models.User and fastify.mongoose, instead?

kill -9 mongoose && rm -rf mongoose
I will leave the action to you, Oliver, but I think we can satisfactorily close this.

Sure, let's focus on Prisma.


rekor-cli, rekor-server: 0.4.0 -> 0.5.0

Motivation for this change
https://github.com/sigstore/rekor/releases/tag/v0.5.0
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

rekor-cli
rekor-server

https://github.com/ofborg build rekor-cli rekor-server


Investigate why built-in miner produced a vout to reward themself with pubkey type instead of pubkeyhash

After talking with Poramin, he concerned about regression. So we need  to careful on this.

Is this still an issue following Sigma?

Yes.

Work on this when we have bandwidth.

Let's re-check on this once core upgrade is done.


added scheduled sampling for batch model

I added scheduled sampling for the batched attention model.
I don't have cuda or sconce set up on my computer, but it runs very slowly on a cpu.

Hi @patrick-dd , couldn't understand your implementation of scheduled sampling. You are creating output_placeholder but you are not using it at all, you are passing the same single batched best word as the next input?
decoder_input = Variable(ni)


Add "Thanks" support back

Is your feature request related to a problem? Please describe.
The previous gitter bot had a "brownie points" feature, which will not be replicated due to spam and "gaming the system" issues. However, having a "hollow" version I think would be acceptable.
Describe the solution you'd like
Anytime someone thanks someone else, the bot will "give brownie points" to the other user. This wont actually increase anyone's points or anything, but will bring recognition to those that help others, which was the main goal of the originals.
Additional context
TDB (I want to add a reference to the originals text/emoji/setup, we will use the same one for this iteration)

I've got some code started for this. If anyone else is working on it let me know! :)


Feat: Use views in relations

close #769

Added types and correct handling of Views to relations and alias
Added correct handling of Views to the dialect for RQB and correct types to the query builder
Added schema for integrations tests with Views and tests for the schema with views to the RQB

Upon merging this PR, the relational query builder will also accept views as well as tables when defining relations.
For example, the following will be possible:
export const usersView = mysqlView('users_view').as((qb) =>
	qb.select({
		id: usersTable.id,
		name: usersTable.name,
		invitedBy: usersTable.invitedBy,
	}).from(usersTable)
);

export const usersConfig = relations(usersView, ({ one, many }) => ({
	invitee: one(usersView, {
		fields: [usersView.invitedBy],
		references: [usersView.id],
	}),
	posts: many(postsView),
}));

export const postsView = mysqlView('posts_view').as((qb) =>
	qb.select({
		id: postsTable.id,
		content: postsTable.content,
		ownerId: postsTable.ownerId,
	}).from(postsTable)
);

export const postsConfig = relations(postsView, ({ one, many }) => ({
	author: one(usersView, {
		fields: [postsView.ownerId],
		references: [usersView.id],
	}),
}));

const usersWithPosts = await db.query.usersView.findMany({
  with: {
    posts: true,
  },
});

And all the relational query API will be available just like they were tables.
The tests that were added are exactly the same as the tests for the regular RQB, as those tests are very complete. They are just using views instead.

We're anxiously awaiting this one.  Thanks @Angelelz!

Same here, any ETA for merging this?

Drizzle's query API is pleasant to work with but feels limited in terms of flexibility and often forces me to fall back to using the more flexible select API. Enabling views to work along with relations would fix most of the limitations I've come across and thus increase usage of the query API. I hope we get to play around with this in the near future!

What can we do to expedite getting this PR released? The suggested workaround of creating “shadow” tables to make relational queries work is not scalable.

Hey any chance this can get merged in now?

Drizzle's query API is pleasant to work with but feels limited in terms of flexibility and often forces me to fall back to using the more flexible select API. Enabling views to work along with relations would fix most of the limitations I've come across and thus increase usage of the query API. I hope we get to play around with this in the near future!

💯% agree


Update Dockerfile

Works good on ubuntu:18.04 and python3.6
After creating your pull request, tick these boxes if they are applicable to you.

[+] I have tested my changes against the review branch (the latest developmental version), and this pull request is targeting that branch as a base
[+] I have tested my changes on Python 3.5.3 or higher

We dont want to fundamentally change the OS

We dont want to fundamentally change the OS

Check your Dockerfile.
It doesn't work with apline


add release note

Hi how about using milestone and add some release note to HikariCP release cycle?
thanks

@brettwooldridge , It would be nice to have some release notes for 3.4.0 or have it in the change log here https://github.com/brettwooldridge/HikariCP/blob/dev/CHANGES. Thanks.


store the selected option

is there any way to store the selected value on state ?? something like an onSelect method which returns the selected option when it changes ?

Yes, there is an onSelect method. Please see the readme API section.


build: add Bazel build system

This is a continuation of #4, which got a little messed up when the repo was made public.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Apologies to anyone tracking for the commit spam. Here's the current status:

All build steps pass locally and on CI. This includes all packaging and release steps through jsii-release
All tests currently build and run correctly. The only errors are due to out-of-date snapshots. Functionality to update the snapshots through Bazel is pending
All examples build and generate the imports files through Bazel correctly, in addition to tests. However example synthesis is still pending (though is mostly done). Though strictly speaking they synthesize correctly, they don't end up in the correct directory for easy use.

tl;dr only two things left: snapshot test updation through Bazel, example synthesis through Bazel

Progress update:

Jest tests all pass and snapshot updation is covered and working. The CLI tests need an NPM patch so that require.resolve will work correctly with JSII
Example synthesis is still pending, but is the only major hurdle left

Progress update:

CLI tests are working fine now. Snapshot updation has a minor issue in the examples, but tests run regardless
Examples synth is still pending
Fixed a packaging issue that preventing yaml from getting bundled with the library. Also added the LICENSE file

All systems go! There is one precondition for doing example synthesis that I'm trying to make easier, but in the meantime the base functionality for the entire repo has been replicated with Bazel in some form.

@CaerusKaru can you do one last merge from master and I'll pick this up

@eladb Should be all set :+1:

I am getting the following message when running "yarn install":
Unrecognized patch file in patches directory rules_nodejs.patch

@CaerusKaru this is not good enough yet... too much duplicated information all over the place. Repeating that one of the main motivations for me is to "DRY"-up all the info that we need to supply in order to build. I want to be able to provide the minimum amount of information and never repeat myself if not needed.

@eladb There is more we can do in terms of compartmentalizing by creating macros, but really it's up to you to determine the ergonomics you want. So far my goal has just been to get the project "working" in an MVP state. How you want it to work after that is really just gravy now that it does work.

Yes I understand. Let me try to play with this a little to see if I can get to the ergonomics I want

@CaerusKaru I think I am going to close this one for the time being. I don't think I have the time in the next few months to spend on getting this to a place that I would feel comfortable merging. @iliapolo would you be interested to look into this perhaps?

@CaerusKaru I really appreciate all the effort!

@el

@eladb definitely. I’ll see if I can pick this up.


Add provider fusionauth

This adds support for FusionAuth as a provider.
https://github.com/theogravity/pulumi-fusionauth
It is bridged using https://github.com/gpsinsight/terraform-provider-fusionauth

Hi @theogravity,
this looks fantastic! thank you so much for this - the one small change I need to ask for is that in your upstream repo you have _docs and we assume the path is docs.  That will help us be able to pull the correct documentation for your provider. When you do that, you can post v0.9.6 and we can get this published
Thanks
Paul

Done!

Wow! thanks for the fast feedback here @theogravity :) This looks good!!

@stack72 When does the registry deploy happen? Unable to find it in the registry.


Introduce Pull Request Reviewers

Introduces a generated set of code owners to ensure a good contribution experience.
For more details/timelines (internal): go/codeowners

@confluentinc/Integration Architecture, @confluentinc/connect, @confluentinc/devx please take action to review and merge this pull request.
For more details/timelines see: go/codeowners
Note: This pull request and subsequent comments are automatically generated

@confluentinc/connect, @confluentinc/devx, @confluentinc/Integration Architecture please take action to review and merge this pull request.
For more details/timelines see: go/codeowners
Note: This pull request and subsequent comments are automatically generated


removing the "reduce terms" components (real estate eventually to be used for "did you mean...")

Codecov Report

Merging #143 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #143      +/-   ##
==========================================
+ Coverage   79.24%   79.27%   +0.03%
==========================================
  Files         117      117
  Lines        7862     7837      -25
  Branches     1195     1191       -4
==========================================
- Hits         6230     6213      -17
+ Misses       1632     1624       -8

Impacted Files
Coverage Δ

app/src/home/home-controller.js
80.95% <ø> (-0.3%)
:arrow_down:

app/src/search/search-input-controller.js
73.04% <ø> (+0.44%)
:arrow_up:

app/src/filters/filters.js
75% <100%> (+2.41%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 774d7c0...0f21494. Read the comment docs.


[Revise subclass] congenital myasthenic syndrome 12 (MONDO:0012518)

Mondo term (ID and Label)
congenital myasthenic syndrome 12 (MONDO:0012518)
Any congenital myasthenic syndromes with glycosylation defect in which the cause of the disease is a mutation in the GFPT1 gene.
should be is_a
is_a
"disorder of glycosylation" (MONDO:0024322)
Suggested revision and reasons
Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here

Dear @ValWood,
Thank you for reaching out to the Mondo Disease Ontology, part of the Monarch Initiative!
We have received your ticket and will review it as soon as possible.
If you have any additional information or updates relevant to your inquiry, please add them in a comment on this ticket. Your input is valuable in helping us understand and resolve the issue effectively.
Thank you for your patience and for your contributions to Mondo!
Sincerely,
The Mondo Team


Tenor Filter

Have the Kakapo bot remove Tenor links from certain channels. Tenor links should be automatically as soon as the user posts them.
The following channels to filter for Tenor links are:
#announcements
#engagement-announcements
#unit-rules
#dev-announcements
Everything in Tutorials & Info
#introduce-yourself
#general
#loa
#suggestions
#operation-feedback
#host-mod-list
#scheduling
#host-general
I do not care about #voice-chat

Testing this against QA. The following tests will be done:
Testing Core Functions

[ ] Posting tenor.com links
[ ] Posting tenor.co links
[ ] Posting Tenor links in
[ ] #announcements
[ ] #engagement-announcements
[ ] #unit-rules
[ ] #dev-announcements

Testing specific channels

[ ] #introduce-yourself
[ ] #general
[ ] #loa
[ ] #suggestions
[ ] #operation-feedback
[ ] #host-mod-list
[ ] #scheduling
[ ] #host-general

Testing roles

Posting by Operations Command
Posting by Command Consultant
Posting by Operator
Posting by Combat Service Center
Posting by Bot

Tests marked with a check have passed the test and will be moved to done.

QA has passed.
Only bug that appeared is that users can save tenor gifs then post them as cdns to bypass the filter. Will fix it later.


arch: move up_irq{save|restore} to irq.h

Summary
Fixes https://github.com/apache/incubator-nuttx/pull/2566#discussion_r616836971
Build breaks with
nuttx/include/nuttx/arch.h:1386:12: error: conflicting declaration of 'irqstate_t up_irq_save()' with 'C' linkage
 1386 | irqstate_t up_irq_save(void);
      |            ^~~~~~~~~~~

Impact
Restores build
Testing
CI

While I agree, I want to get to the bottom of why this caused issues for you and none of the other builds so that we can prevent it from breaking again.

@btashton I agree!
Here I what I can say...
I was working on up-taking NuttX master  for PX4 and it showed up. (lost in 1000s of format specifier issues)
We have -Wfatal-errors and some driver code in PX4 do include arch.h and I would assume irq.h, but the error does not tell me where the conflicting definition was.
I have several changes to upstream for the format specifier issues, then I will push the PX4 build on NuttX master ish....
What else would help you?

@gustavonihei  @btashton  - how should we proceed?

@gustavonihei @btashton - how should we proceed?

I believe that those function declarations on bl602/irq.h are not necessary and should be removed. Those function are already declared on the RISC-V generic header, which is shared among other chips.
If removing them triggers another error, then we should fix the root cause.

@gustavonihei @btashton - how should we proceed?

I believe that those function declarations on bl602/irq.h are not necessary and should be removed. Those function are already declared on the RISC-V generic header, which is shared among other chips.
If removing them triggers another error, then we should fix the root cause.

@gustavonihei - thank you. Let's See how CI feels about it.


Avoid eprintln in the library

Printing stuff to stdout is unsuitable for a GUI.
The simplest solution could be to use the log crate, and warn!()/error!() as appropriate.

Agreed!


Annoying Warning ( CTCTextEncoder)

Describe the bug
When running recipes that involve CTC, we always output the following warning:

/scratch/ravanelm/speechbrain/speechbrain/dataio/encoder.py:722: UserWarning: CTCTextEncoder.expect_len was never called: assuming category count of 40 to be correct! Sanity check your encoder using `.expect_len`. Ensure that downstream code also uses the correct size. If you are sure this does not apply to you, use `.ignore_len`.
  warnings.warn(

This can be useful, but it is very annoying if it comes out every time. Aku, do you see a fix for that?
Expected behaviour

To Reproduce

Versions

Relevant log output
-

Additional context

@AsuMagic

@Gastron, any insight on that? It is pretty annoying as it appears every time we run a recipe with CTC.

Similarly, we often see this warning: "UserWarning: CategoricalEncoder.expect_len was never called: assuming category count of 4 to be correct! Sanity check your encoder using .expect_len. Ensure that downstream code also uses the correct size. If you are sure this does not apply to you, use .ignore_len.". How can we avoid that to happen when running a recipe?

It looks like it is a change you made @AsuMagic . This causes many unnecessary warnings while running some recipes (e.g,, TIMIT)

Yes, the change was intentional to prevent accidental mismatches. At the time of the PR I didn't update recipes (partly because IIRC recipe tests weren't that ready back then). Will need to look into it.

I don't think I will have the time to update all recipes given the work time I have left (on top of existing PRs+streaming), so I spent some time thinking about how the warning should be improved instead. Right now, it looks like:

CTCTextEncoder.expect_len was never called: assuming category count of 1000 to be correct! Sanity check your encoder using .expect_len. Ensure that downstream code also uses the correct size. If you are sure this does not apply to you, use .ignore_len.

I had some ideas for shorter messages but discussing it with an (uninvolved) developer, it seems to me that the verbosity remains useful/necessary and that my attempts at shorter warnings were not better.
So, would just revising the tone of the first sentence and adding a notice in the warning that states "This may be normal, users can ignore this." suffice?

any updates @mravanelli ?


Multiple diagnostic messages emitted for duplicate definition

With reference to the common issue #18361, and following the recommendation in tests/compiler/dart2js/message_kind_helper.dart to create an issue on this situation:
Duplicate declarations of initializing formal parameters give rise to multiple diagnostic messages with dart2js --initializing-formal-access. For instance, consider the following program:
class C {
  var x;
  C(this.x, this.x);
}

main() {
  print(new C(2, 2));
}

The program causes the following messages:
Error: Field 'x' is initialized more than once.
Info: 'x' was already initialized here.
008.dart:6:18:
Error: Duplicate definition of 'x'.
Try to rename or remove this definition.
  C(this.x, this.x);
                 ^
008.dart:6:10:
Info: Existing definition of 'x'.
  C(this.x, this.x);
         ^
Error: Compilation failed.

Emitting messages about just one of the problems caused by the duplication would be less confusing for developers. No information would be lost since there is only one root problem: the duplication.
Note that the effect of having option --initializing-formal-access will be the regular semantics soon, i.e., the issue will then exist also for runs with no options.

Turns out the problem still exists.


Allow user to specify a custom Logger

Prosopite will default to Rails.logger but you can provide your custom
logger instance in case you want to change how or where errors
are logged.
(implements #8)

Thanks for the PR!
Instead of replacing all loggers with a new one, i think that the custom logger should be added as a new Prosopite.custom_logger option and the existing rails / stderr loggers should log using red color by default.

I've thought on adding yet another log option to the config, but the reason that drove me away from it is 2 folded:

if we add red by default, then someone may come and say "hey I want blue now" so we would start adding configurations for logger details, which is something I would try to avoid.
By replacing all loggers with the user provided one and sensible defaults we can provide a lot of flexibility while reducing the complexity of the configuration. If anyone wants to use multiple logging destinations, perhaps they can leverage ActiveSupport::Logger like:

require "active_support/logger"

console_logger = ActiveSupport::Logger.new(STDERR)
file_logger = ActiveSupport::Logger.new("prosopite.log")
combined_logger = console_logger.extend(ActiveSupport::Logger.broadcast(file_logger))

Prosopite.logger = combined_logger

Let me know what you think!

For me, given the nature of the problem, it makes more sense to log using
red on rails log / stderr by default. If a user wants other colours or file
locations then Prosopite.custom_logger can do the job. I also prefer the
existing boolean options for these file locations instead of having to
write logger code for doing the same.
Στις Τετ, 3 Μαρ 2021, 5:07 μ.μ. ο χρήστης F-3r<EMAIL_ADDRESS>έγραψε:

I've thought on adding yet another log option to the config, but the
reason that drove me away from it is 2 folded:

if we add red by default, then someone may come and say "hey I want
blue now" so we would start adding configurations for logger details, which
is something I would try to avoid.
By replacing all loggers with the user provided one and sensible
defaults we can provide a lot of flexibility while reducing the complexity
of the configuration. If anyone wants to use multiple logging destinations,
perhaps they can leverage ActiveSupport::Logger like:

require "active_support/logger"
console_logger = ActiveSupport::Logger.new(STDERR)file_logger = ActiveSupport::Logger.new("prosopite.log")combined_logger = console_logger.extend(ActiveSupport::Logger.broadcast(file_logger))
Prosopite.logger = combined_logger
Let me know what you think!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/charkost/prosopite/pull/9#issuecomment-789781780, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAG2TMTDKGZ7RMFVTLC6FZTTBZGCPANCNFSM4YQQMK4Q
.


Screen and tab change stopped working on web using react-native-web

Works perfect on mobile (so not sure if you guys are gonna pay attention to this)
Bug only happens Web (I'm using react-native-web)
Redux integration

Current Behavior
When I dispatch a new navigation or reset action, the redux detects it correctly, but the screen does not change. It also happens with the tab bar, I click to change tab and it does not change

PS1: This tab bug happens only in the last version, beta 9, not on beta 8
PS2: The screen not changing already happened in beta 8

It's weird because it works fine on mobile and the code is the same.
Any insights on what can be causing this? Could you indicate to me the lines of code responsible to render the new screen on change?

Expected Behavior
Screen change after redux action or tab press.
Your Environment

software
version

react-navigation
1.0.0-beta.9

react-native-web
0.0.94

react-native
0.43.2

node
7.9.0

npm or yarn
yarn 0.23.4

I'm having this same issue.

@smkhalsa glad to know. what's your environment? mobile? normal web? react-native-web? since when you facing this problem?

@brunolemos similar setup as you. Running react-native (works fine) and react-native-web (freezes after tab change).
"react": "16.0.0-alpha.3",
"react-native": "0.43.2",
"react-native-web": "^0.0.94",
"react-navigation": "^1.0.0-beta.9",
I only just implemented react-native-web, so I don't have any experience with prior version of react-navigation.

@smkhalsa Ok, thanks. Try making it work on 1.0.0-beta.8 first, because mine does not freeze, it just doesn't happen anything.

I should have been more specific. The app is not freezing - rather the screen doesn't change (as you mentioned above). If I look at my redux dev tools, I see that actions are still being dispatched and the navigation state is changing.
On interesting fact is that it DOES change tabs when I first tap a new tab, but every successive tap to a tab does nothing.

On interesting fact is that it DOES change tabs when I first tap a new tab, but every successive tap
to a tab does nothing.

Yes I notice a similar thing with a reset or navigate screen action. The first navigation change seems to work just fine, the next ones doesn't.

So if I understand correctly, the navigation state is fine, but the view stops responding after a tab change?
Sounds like a bug with tab view, which @satya164 has more context on.
I'm not sure how it would be configured, but it might make sense to have a different tab view implementation for web (we could still use the TabRouter, and the API would be the same, but it sounds like the RN implementation of the view is causing troubles.)

@ericvicenti I don't believe the issue is unique to tab view changes. That happens to be the component I'm using, but based on @brunolemos's comments above, it the navigation does not respond after the first navigation event, regardless of the component used.

Ah, then maybe we can pin down an issue in the router! Is there a specific previous state +action that causes the bug to happen? You can add logging around the router.getStateForAction to see. It should be easy to fix if we can find a broken test case to add to our suite!

Is there a repo somewhere we can check?

@satya164 I haven't created a minimal code for reproduction yet, but I just added you and @ericvicenti to the project if you want to see the source code and see it happening.

Did anyone find anything? cc @smkhalsa
I don't get this error but possibly related: #1660

Sorry I've not gotten time to check it out yet :(

I also haven't had time to dig deeper into this yet.
On Tue, May 23, 2017 at 10:31 PM Satyajit Sahoo<EMAIL_ADDRESS>wrote:

Sorry I've not gotten time to check it out yet :(
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/react-community/react-navigation/issues/1407#issuecomment-303615791,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGEXF_wJt9btQix6hodmEwFeRAUFU_05ks5r87SggaJpZM4NTM-z
.

--
Sat Mandir Singh Khalsa
<EMAIL_ADDRESS>+1<PHONE_NUMBER>

@satya164 My boilerplate has an @next branch you could use to try this out https://github.com/ericwooley/react-nativeish/tree/next
one thing to take note of when testing this is, web/webpack.dev.js aliases react-navigation,
'react-navigation': 'react-navigation/lib-rn/react-navigation.js'

I found a problem that's is much likely causing this issue:
setState callbacks are not being called at all.

If this is indeed the problem, it's not directly related to react-navigation

And somehow this is caused by my build setup (I'm using create-react-app). setState callbacks that are inside my web/src folder works just fine. The ones outside are not triggered. Any one has any idea what could cause this?
@smkhalsa Could you check if you have this problem too? Replace one of your components with this code and check if it will console.log the callbacks correctly or not (mine doesn't, depending on the folder I put this):
import React from 'react'

export default class App extends React.PureComponent {
  constructor(props) {
    super(props)
    this.state = { count: 0 }
  }

  componentDidMount() {
    console.log('handlePress 1/3')

    this.setState(
      ({ count }) => ({ count: count + 1 }),
      () => {
        console.log('handlePress 2/3 (callback)')
      },
    )

    this.setState({ a: true }, () => {
      console.log('handlePress 3/3 (callback)')
    })
  }

  render() {
    return null
  }
}

Hey guys, found the solution! 🎉
Downgrading React to 15.5.4 fixes all issues (the navigation not changing, etc).
Apparently setState callbacks are not being called when using react >= 16.
Haven't checked yet if it's a problem on React or a incompatibility of react-native-web with react 16. cc @necolas

RNW Readme clearly states that you need to use React 15

@necolas subtle as always lol
Only thing I see about react version in the readme is the install line: npm install --save<EMAIL_ADDRESS>If it's only that, sorry but it's not clear at all that it doesn't support react 16. At most it makes people think the readme wasn't updated.
But nevermind, the problem here is that I THOUGHT I was using react 15. But when I imported a file from outside my web src folder, it started pointing to the mobile node_modules, which uses and requires react 16.
I'll probably have to make something like learna works (had problem before with multiple react instances error), or if anyone has another solution (webpack changes, ...) please let me know.

You were probably also seeing peerDependency warnings